feat(mcp): add windsurf host to init-client config - #2664
Conversation
Add `windsurf` to `gittensory-mcp init-client --print`. Windsurf reads ~/.codeium/windsurf/mcp_config.json using the same shared `mcpServers` JSON shape as claude/cursor/mcp, so it reuses that snippet branch (unlike vscode's `servers`/`type` shape). Updates the CLI usage text, the empty-client and unsupported-client error messages, the README example + host description, and the public MCP-clients docs page (docs.mcp-clients.tsx) generate-config command list so the docs match the new CLI surface. A test asserts the windsurf snippet equals the shared mcpServers snippet. No issue because issue creation is restricted on this repo; this is a small additive host mirroring the vscode host addition (JSONbored#1770), no schema/API change.
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-03 02:36:10 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 4 non-blocking
Review context
Contributor next steps
Signal definitions
🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2664 +/- ##
=======================================
Coverage 96.10% 96.10%
=======================================
Files 237 237
Lines 26540 26540
Branches 9625 9625
=======================================
Hits 25507 25507
Misses 424 424
Partials 609 609 🚀 New features to boost your workflow:
|
JSONbored
left a comment
There was a problem hiding this comment.
Please fix:
The only notable gap is documentation depth on the docs page: it lists the command but does not tell users the Windsurf config filename there.
Summary
Adds
windsurftogittensory-mcp init-client --print, so Windsurf users can generate their MCP config like the other supported hosts.Windsurf reads
~/.codeium/windsurf/mcp_config.jsonusing the same sharedmcpServersJSON shape asclaude/cursor/mcp, so it reuses that snippet branch (no new snippet, unlikevscode'sservers/"type": "stdio"shape).Mirrors the merged
vscodehost addition (#1770): CLI usage text, the empty-client and unsupported-client error messages, the README example + host description, and — for docs parity — the public MCP-clients docs page (docs.mcp-clients.tsx) generate-config command list. A test asserts thewindsurfsnippet equals the sharedmcpServerssnippet.No issue because issue creation is restricted on this repo for outside accounts; this is a small additive host with no schema or API change.
Scope
Validation
git diff --checknpm run typechecknpm run build:mcp(node --checkclean)npm run ui:typecheck(clean)npm run test:coverage(scoped:test/unit/mcp-cli-basics.test.ts— 20 pass, incl. the addedwindsurfsnippet assertion; CLI exercised by spawninginit-client --print windsurf)npm run test:mcp-pack/ui:lint/ui:build(see note)npm audit --audit-level=moderateIf any required check was skipped, explain why:
docs.mcp-clients.tsxgenerate-config command list.build:mcpandui:typecheckpass;prettier --checkon the changed docs file passes;mcp-cli-basics.test.tspasses (20). The MCP-package JS is outside Codecov'ssrc/**/*.tsinclude (no patch-coverage obligation). I could not runtest:mcp-packlocally (Node 24 vs the repo's pinned Node 22 —check-mcp-package.mjsthrowsERR_INVALID_ARG_TYPE, identically on unmodifiedmain), andui:lint/ui:buildare dominated by pre-existing CRLF noise on this Windows checkout (core.autocrlf); CI runs both on a clean LF, pinned-Node checkout.Safety
Notes
docs.mcp-clients.tsxgenerate-config list now includeswindsurfalongside codex/claude/cursor/mcp.